Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New Adapter ZEDO #2872

Merged
merged 11 commits into from
Jul 23, 2018
Merged

New Adapter ZEDO #2872

merged 11 commits into from
Jul 23, 2018

Conversation

skazedo
Copy link
Contributor

@skazedo skazedo commented Jul 20, 2018

Type of change

  • Bugfix
  • Feature
  • [X ] New bidder adapter
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Does this change affect user-facing APIs or examples documented on http://prebid.org?
  • Other

Description of change

  • test parameters for validating bids
{
  bidder: 'zedo',
  params: {
    code: 2264004118
    dimId: 9
  }
}

Be sure to test the integration with your adserver using the Hello World sample page.

For any changes that affect user-facing APIs or example code documented on http://prebid.org, please provide:

Other information

Copy link
Collaborator

@snapwich snapwich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code looks okay, but it's missing code to handle the GDPR string for the auction request and syncs.

@skazedo
Copy link
Contributor Author

skazedo commented Jul 23, 2018

@snapwich, I have commited changes to pass the gdpr consent in the request and sync .

Copy link
Collaborator

@snapwich snapwich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@snapwich snapwich merged commit 8eedfc3 into prebid:master Jul 23, 2018
mkendall07 added a commit that referenced this pull request Jul 24, 2018
snapwich pushed a commit that referenced this pull request Jul 24, 2018
* Revert "New Adapter ZEDO (#2872)"

This reverts commit 8eedfc3.

* Revert "don't allow null or undefined bid properties (#2865)"

This reverts commit f5170d9.
florevallatmrf pushed a commit to Marfeel/Prebid.js that referenced this pull request Sep 6, 2018
* initial commit

* updated contact and tag details

* changes ti support the renderers

* changes to pass dimId

* fixed names of internal mapping

* added comment

* added gdpr param to request and other fixes
florevallatmrf pushed a commit to Marfeel/Prebid.js that referenced this pull request Sep 6, 2018
* Revert "New Adapter ZEDO (prebid#2872)"

This reverts commit 8eedfc3.

* Revert "don't allow null or undefined bid properties (prebid#2865)"

This reverts commit f5170d9.
StefanWallin pushed a commit to mittmedia/Prebid.js that referenced this pull request Sep 28, 2018
* initial commit

* updated contact and tag details

* changes ti support the renderers

* changes to pass dimId

* fixed names of internal mapping

* added comment

* added gdpr param to request and other fixes
StefanWallin pushed a commit to mittmedia/Prebid.js that referenced this pull request Sep 28, 2018
* Revert "New Adapter ZEDO (prebid#2872)"

This reverts commit 8eedfc3.

* Revert "don't allow null or undefined bid properties (prebid#2865)"

This reverts commit f5170d9.
ghost pushed a commit to devunrulymedia/Prebid.js that referenced this pull request Jan 30, 2019
* initial commit

* updated contact and tag details

* changes ti support the renderers

* changes to pass dimId

* fixed names of internal mapping

* added comment

* added gdpr param to request and other fixes
ghost pushed a commit to devunrulymedia/Prebid.js that referenced this pull request Jan 30, 2019
* Revert "New Adapter ZEDO (prebid#2872)"

This reverts commit 8eedfc3.

* Revert "don't allow null or undefined bid properties (prebid#2865)"

This reverts commit f5170d9.
AlessandroDG pushed a commit to simplaex/Prebid.js that referenced this pull request Mar 26, 2019
* initial commit

* updated contact and tag details

* changes ti support the renderers

* changes to pass dimId

* fixed names of internal mapping

* added comment

* added gdpr param to request and other fixes
AlessandroDG pushed a commit to simplaex/Prebid.js that referenced this pull request Mar 26, 2019
* Revert "New Adapter ZEDO (prebid#2872)"

This reverts commit 8eedfc3.

* Revert "don't allow null or undefined bid properties (prebid#2865)"

This reverts commit f5170d9.
AlessandroDG pushed a commit to simplaex/Prebid.js that referenced this pull request Mar 26, 2019
* initial commit

* updated contact and tag details

* changes ti support the renderers

* changes to pass dimId

* fixed names of internal mapping

* added comment

* added gdpr param to request and other fixes
AlessandroDG pushed a commit to simplaex/Prebid.js that referenced this pull request Mar 26, 2019
* Revert "New Adapter ZEDO (prebid#2872)"

This reverts commit 8eedfc3.

* Revert "don't allow null or undefined bid properties (prebid#2865)"

This reverts commit f5170d9.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants